feat: add AWS Lambda Managed Instances (LMI) skill to aws-serverless plugin#140
Open
sbbhimji wants to merge 11 commits into
Open
feat: add AWS Lambda Managed Instances (LMI) skill to aws-serverless plugin#140sbbhimji wants to merge 11 commits into
sbbhimji wants to merge 11 commits into
Conversation
…plugin Add a new skill for evaluating, configuring, and migrating workloads to AWS Lambda Managed Instances. Includes workload fitness assessment, 4-column cost comparison (Lambda OD/SP vs LMI OD/SP), configuration recommendations, thread-safety review, and end-to-end migration framework. Reference files cover cost analysis, configuration tuning, thread safety, runtime-specific migration patterns, infrastructure setup (CLI/SAM/CDK), and troubleshooting.
…nces - Add managed-instances and lmi keywords to plugin.json - Add LMI skill triggers to README aws-serverless section - Add cross-reference to LMI skill in aws-lambda SKILL.md (key capabilities and "When to Load Reference Files" sections) - Update plugin description to mention Lambda Managed Instances
sbbhimji
requested review from
a team,
krokoko,
scottschreckengaust and
theagenticguy
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new aws-lambda-managed-instances skill to the aws-serverless plugin, providing an agent workflow and reference material to evaluate, configure, and migrate workloads to Lambda Managed Instances (LMI), and wires it into existing plugin docs/metadata.
Changes:
- Introduces a new
aws-lambda-managed-instancesskill (SKILL.md) plus several reference guides (cost, configuration, thread safety, migration patterns, infra setup, troubleshooting). - Updates the existing
aws-lambdaskill to cross-reference the new managed-instances skill. - Updates plugin metadata and root README to mention LMI and add trigger phrases/keywords.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| plugins/aws-serverless/skills/aws-lambda/SKILL.md | Adds cross-references so the Lambda skill routes LMI questions to the new skill. |
| plugins/aws-serverless/skills/aws-lambda-managed-instances/SKILL.md | New core workflow and routing to reference files for LMI evaluation/config/migration. |
| plugins/aws-serverless/skills/aws-lambda-managed-instances/references/cost-comparison.md | Adds pricing formulas, comparison template, and worked example for Lambda vs LMI. |
| plugins/aws-serverless/skills/aws-lambda-managed-instances/references/configuration-guide.md | Adds instance/memory/concurrency tuning guidance and scaling controls. |
| plugins/aws-serverless/skills/aws-lambda-managed-instances/references/thread-safety.md | Adds a thread-safety checklist and runtime-specific guidance. |
| plugins/aws-serverless/skills/aws-lambda-managed-instances/references/migration-patterns.md | Adds before/after migration examples for multi-concurrency readiness. |
| plugins/aws-serverless/skills/aws-lambda-managed-instances/references/infrastructure-setup.md | Adds IAM/VPC/CLI + SAM template guidance for capacity provider setup. |
| plugins/aws-serverless/skills/aws-lambda-managed-instances/references/troubleshooting.md | Adds common issues table and debugging steps. |
| plugins/aws-serverless/.claude-plugin/plugin.json | Adds managed-instances and lmi keywords for discoverability. |
| README.md | Updates plugin description and adds the new skill to the aws-serverless trigger table. |
bfreiberg
requested changes
May 3, 2026
…model - Fix Python concurrency model: process-based isolation, no thread safety needed - Clarify cold starts claim (provisioned capacity only) - Replace hardcoded regional availability with docs link - Replace LocalStack reference with generic non-prod testing - Convert Step 5 to numbered procedure - Simplify cost comparison (defer SP/RI to pricing calculator) - Use least-privilege IAM with ec2:ManagedResourceOperator condition - Add setup-lmi.sh script for automated provisioning - Add VPC endpoint requirements table with costs - Add Powertools compatibility and SDK minimum versions - Add CloudWatch metrics dimensions guidance - Add pricing calculator, samples repo, and migration blog to resources
leandrodamascena
suggested changes
May 9, 2026
leandrodamascena
left a comment
leandrodamascena
left a comment
There was a problem hiding this comment.
Hey @sbbhimji, nice work on this, the skill is well structured and the content is technically accurate. Left 2 small comments to address before merging.
…d cost example - Update Powertools .NET to supported (logging, tracing, idempotency, batch, parameters) - Remove hardcoded cost example, defer to LMI Pricing Calculator
bfreiberg
reviewed
May 11, 2026
bfreiberg
reviewed
May 11, 2026
bfreiberg
reviewed
May 11, 2026
bfreiberg
reviewed
May 11, 2026
…pricing - Strip cost-comparison.md to just pricing calculator links - Remove cost column from VPC endpoints table in infrastructure-setup.md
sbbhimji
force-pushed
the
feature/aws-lambda-managed-instances
branch
from
39a9417 to
12a006b
Compare
Merge upstream main — keep both LMI description update and new codebase-documentor-for-aws and migration-to-aws plugins.
…ces/headings/lists)
leandrodamascena
approved these changes
May 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a new skill to help users evaluate, configure, and migrate workloads to AWS Lambda Managed Instances (LMI) — the
recently launched feature that runs Lambda functions on dedicated EC2 instances with AWS managing the lifecycle.
What's included
SKILL.md (205 lines) — core workflow with 6 steps:
6 reference files (42-96 lines each):
cost-comparison.md— pricing formulas, discount comparison, worked example, break-even guidanceconfiguration-guide.md— instance type decision tree, memory sizing from existing Lambda/EC2, concurrency tuning,scaling controls
thread-safety.md— code review checklist, runtime-specific guidance (Node.js, Python, Java, .NET), commonanti-patterns
migration-patterns.md— before/after code examples per runtime (global state, file I/O, database connections)infrastructure-setup.md— IAM roles, VPC requirements, CLI workflow, SAM templatetroubleshooting.md— common issues, debugging steps for startup/performance/costPlugin registration:
managed-instancesandlmikeywords toplugin.jsonaws-serverlesssectionaws-lambda/SKILL.md(Key Capabilities + When to Load Reference Files)Design decisions
Compute Savings Plans covering Lambda duration
this is the feat: initial branch #1 source of production issues when adopting LMI
progressive disclosure, explicit defaults, error handling
Test plan
claude --plugin-dir ./plugins/aws-serverlessaws-lambdaskill routes to LMI skillclaude --plugin-dir ./plugins/aws-serverless --verboseto verify < 5000 token initial loadBy submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.